Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Waleed Networkx integration #1456

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

feat: Waleed Networkx integration #1456

wants to merge 3 commits into from

Conversation

waleedalzarooni
Copy link
Collaborator

Description

I have integrated Networkx, an additional graph storage method. This method allows users to create graph relationships without an external database being involved providing versatility to users.

Motivation and Context

Provides additional choices for graph creation
#1373

  • [ X] I have raised an issue to propose this change (required for new features and bug fixes)

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • [X ] New feature (non-breaking change which adds core functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation (update in the documentation)
  • Example (update in the folder of example)

Implemented Tasks

  • [ X] Sympy Implementation
  • [ X] Networkx implementation

Checklist

Go over all the following points, and put an x in all the boxes that apply.
If you are unsure about any of these, don't hesitate to ask. We are here to help!

  • [ X] I have read the CONTRIBUTION guide. (required)
  • [ X] My change requires a change to the documentation.
  • [X ] I have updated the tests accordingly. (required for a bug fix or a new feature)
  • I have updated the documentation accordingly.

In this merge I have integrated Sympy into CAMEL's toolkit folder, this will allow Agents to perform more complex mathematical
operations such as algebraic manipulation, calculus and linear algebra
@waleedalzarooni waleedalzarooni added this to the Sprint 20 milestone Jan 16, 2025
@Wendong-Fan Wendong-Fan requested review from X-TRON404, Asher-hss, Wendong-Fan and Aaron617 and removed request for X-TRON404 January 17, 2025 15:55
@Asher-hss
Copy link
Collaborator

Thanks @waleedalzarooni Could you create a few examples that use these new features?

@waleedalzarooni
Copy link
Collaborator Author

Thanks @waleedalzarooni Could you create a few examples that use these new features?

Sure, would you like it to be done as an implementation in the unit tests or something separate

@waleedalzarooni
Copy link
Collaborator Author

Thanks @waleedalzarooni Could you create a few examples that use these new features?

Sure, would you like it to be done as an implementation in the unit tests or something separate
Also I have not included an implementation for other types of graphs, let me know if my current implementation is the correct standard so that I can include the other graph types

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants